I gladly admit this looks a bit shady :-)

However, as all the internals of `tFunc` run synchronously this should not cause any harm.


changed


Will test and merge but I'm not a fan of having isTemperatureValueFeasible take a box type (Float). I think changing it requires further ranging changes, so if you need to touch that code anyway we can change it to submitting e.g. NaN instead of null, otherwise let's just leave it for when we touch that code again..
Typo in the commit message: bellow -> below


Will have a look at fixing that.


for heruku deployments


d


+1

The code here is outdated and it does not make much sense to maintain it in two different places


That's tedious! Yes, looking at the docs, I assume that Jenkins initiates everything as (or at least via) a service, so a console won't be allocated.

The test is a useful one to have (I'm sure I remember at least one bug in the distant past with Windows CTRL+C handling) - which of these two fixes would be preferable?

1. Alter signal.precheck to determine whether the testsuite is being executed with a Windows console and so only skip the test if the console isn't available
2. Alter signal.ml test to use AllocConsole to ensure it has a console (this is perfectly safe in a service - the Console gets sent to a non-visible desktop and would terminate with the process.


Would it be an idea to avoid com.sun internal classes?
If java8 is acceptable: https://docs.oracle.com/javase/8/docs/api/java/util/Base64.html - or commons codec: https://commons.apache.org/proper/commons-codec/apidocs/org/apache/commons/codec/binary/Base64.html


this can be removed completely


:tada: :balloon: :+1: :100: :ship: :it:


To save byets/effort I'm intentionally (potentially) modifying the options object - by creating an `undefined` `array` property where none existed before.

This will only break things if any external code relies on `if ('array' in options)` checks to be false.

If this causes reported real-world problems, we could easily fix that by adding a tad more bureaucracy. :-)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13436](https://teamcity.mesosphere.io/viewLog.html?buildId=117330&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:09:01


110568302-FixCrashInZoneComponentLoadSummary (JasonGlazer) - x86_64-MacOS-10.9-clang: Tests Failed (1734 of 1996 tests passed, 55 test warnings)

 * 533 tests had: AUD diffs.
 * 188 tests had: EIO diffs.
 * 262 tests had: Table big diffs.
 * 6 tests had: RDD diffs.
 * 2 tests had: ESO small diffs.
 * 2 tests had: MTR small diffs.
 * 3 tests had: Table small diffs.
 * 2 tests had: ESO big diffs.
 * 2 tests had: MTR big diffs.
 * 1 test had: ERR diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-86.87%25-red.svg)


for now we unfortunately need `--no-check-certificate` here. See: https://github.com/TYPO3/CmsComposerPackageGenerator/blob/master/src/Classes/PackagesTYPO3ExtensionsGenerator.php#L31


The `PhotoSet` would reject anything that wasn't a JPEG (or a hash or uploaded file), but yes, will put a check in here too to save it the bother.


Merged as e98d546dc2e946f99d616c11615473aa57e21d0a


I don't quite get this one... could you explain?


You are welcome!


Похоже, тут все варианты ухода от первоначального варианта приводят к синтакцическуму оверхеду, аналогичному оверхеду оригинала.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Rather than having the `fractional_sleep 0.1`, could we not have some kind of positive confirmation along the lines of `wait_for_status`?


可以忘记密码了丫


Not exactly the same @victorfisac, `HideCursor()` only makes the cursor invisible, `DisableCursor()` disables the cursor; `ShowCursor()` and `EnableCursor()` are the same but they have sense to keep coherence between the other ones.


@raysan5 maybe, but then I noticed some raylib's matrix functions have angle parameter come first too and I thought maybe it's your design choice...


This should be
`DataTable *table = table_pair->table;
TableSchema *schema = table_pair->schema;`


Please, delete this line, because is not cool.


Hi @racarone , I started seeing this issue: after launch screen, the screen goes black for a second before showing the stage.  This change is causing it.  Doesn't make sense to me.  Are you able to repro what I'm seeing?  I'm using a .storyboard as my launch screen instead of images.


Sorry about this. You can go back to defining this with its type in the loop scope as before (not sure if it matters to the compiler).


Oops! This was released already, so, need to change it.


seems like `response_body` is only used by `successful_response` but they both are not used in this context. so this declaration is useless :bowtie:


My objective is make raylib as much intuitive as possible for students, I will take a look at the convention I follow and I'll make all functions consistent with it.


Я бы вообще удалил эту строчку


Sorry for the extreme delay! Looks like you've done a great job - just include some more description of what's occurring when you're using a code.


@raysan5 exactly my thoughts :)

@victorfisac I though so too until few months before I discovered when you hide cursor and do camera control, for example, if you move mouse fast enough you can see cursor escaping window borders to the desktop and becomes visible before it gets back to the center of the window... With disable cursor feature you can don't even bother to set cursor back to the center of the screen: it assumes that the window now have infinite area and you can just get current mouse position, subtract previous from it and get the difference like I did here: https://github.com/procedural/quatFirstPersonCamera/blob/master/examples/raylib_opengl3_example/main.c

Hope it makes sense!


Are we leaving the `limit` to the front end? Because we are getting the all `tags` in different ordering. Also, could the `get_tags_that_are` be a scope, so we can chain the `limit` to it?


@alsotang Can you please release this fix? I have the same issue.


ProtoTestEventWrapper lays in another package


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1509 of 1509 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.7%25-green.svg)


I rebased this commit,to eliminate unnecessary comments and mistakenly edited code.


It's better to move this code to createKotlinAfterJavaTask


We should add this constraint from the man page, since only O_NONBLOCK can be set after the queue has been intialized

> The only attribute that can be modified is the setting of the O_NONBLOCK flag in mq_flags


Oh git strikes again!


You can spare a few lines by writing this instead:

```ruby
def remote_helper_path
  @remote_helper_path ||= @system.run_command(
    # Expand Machinery::HELPER_REMOTE_PATH on remote machine
    "bash", "-c", "echo -n #{File.join(Machinery::HELPER_REMOTE_PATH, "machinery-helper")}",
      stdout: :capture
  )
end
```


удалить


просто pt sans narrow, bold это уже стиль начертания - жирный - bold или в цифрах 700 + после основного нужно указать безопасный шрифт из системных из того же семейства что и основной (в дннмо случае sans-serif), например Аrial


It seems it's managed here:
https://repository.jboss.org/nexus/content/groups/public/org/jboss/spec/jboss-javaee-7.0/1.0.0.Final/jboss-javaee-7.0-1.0.0.Final.pom

I don't import this one, though. Perhaps I should?


Yes, yes. Can you make them before sunday?


I thought about it, but it is a POSIX constraint, or a Linux constraint ?


It's ok to give people like me tasks like this... I think the majority of those changes were in my tests! :tongue:


Passed quick tests: 1040 - 29 = 1011


delete commented code!


@pb00068

I want to see how this fares at STC...
If it passes, I plan to merge this with SE-CM for LTC.


See issue #118


return   Math.round(sum * Math.pow(10, 2)) / Math.pow(10, 2);


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1261](http://teamcity.monogame.net/viewLog.html?buildId=14139&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:29:31


Leave ro.opengles.version prop as is, HWC still fully working. This ro.kernel.qemu thingy cause HWC issue.


A couple of things:
You know how I feel about brackets:
```c++
void AutonomousPeriodic()
{
...
}
```
Should be:
```c++
void AutonomousPeriodic(){
...
}
```

Instead of using
```c++
const std::string autoNameDefault = "Default";
const std::string autoNameCustom = "My Auto";
```

In robot.cpp, maybe we should have a class/header with just constants, and a way to access all constants (this way we can pass them between objects easily).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13437](https://teamcity.mesosphere.io/viewLog.html?buildId=117332&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:12


I had a call with @danielmarbach Having ability to access the `IMessageSerializer` from the core will require additional work but it's doable.
The only doubt that I have is the JsonSerializer previously registered by transport. How should we provide the same behavior without transport being able to set the serializer?


looks ok


Ok, confirmed
But... ARMv7 apps dissappeared from play store :(


for my own knowledge, is there an advantage of using a dict here instead of a list?


変更部分について確認しました。


Shouldn't this be "the upcoming variant standard**,** because"?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13438](https://teamcity.mesosphere.io/viewLog.html?buildId=117331&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:08:55


I know why. Look at this frameworks/native/opengl/libs/EGL/Loader.cpp, line 86, It does not init prop memory so atoi(prop) will always return to true and assumes that we are running in an emulator!!!!!!!!!


What was that?


Why CM does this to us?


Partly there is. MSP, SerialRx, Telemetry etc etc are configurable to connect to any interface, including VCP/USB. ESC passthrough should be handled the same. Maybe connected to MSP, not an entry of it's own.
But this is a general thing, has nothing really to do specifically with LUX and this PR/commit. Same for all targets. So, I'm a bit OT here. Sorry.


AddWarningWhenDElightAndShading (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Do you think it'd be worth validating the option used and then using that in lieu of hard coding 'delete'?  Then we could support using other operations of our choosing...


I know there was talks/plans about this, but has it come though ?


Added DB integration to Show Route


@MartinSeeler that looks way better ...


А если кто-то захочет собрать себе версию под Тизен? Пускай пока поваляется.


I'd like to suggest to include most parts of http://blog.ploeh.dk/2015/01/15/10-tips-for-better-pull-requests/ in the "Creating Pull Requests:" part of this file, maybe creating a dedicated page linking to the source and keeping a copy in case the blog article goes offline


As you can see on the diff there was no loop before and I don't think that the loop makes sense, as it may run the test forever.

Whether or not we have the data ready was figured out by assertArrayEquals, because if the data was not ready within 5 seconds then the array was null.

The problem with the test is that either there are too many random packets failures to transfer the data within 5 seconds and eventually the timeout should be increased or it's possible that something goes wrong(a bug) and no matter how big timeout we give it will fail. But I have not investigated that yet.


Yes, I think the test was failing after 5 seconds.


This is due to http://www.rubydoc.info/github/bbatsov/rubocop/Rubocop/Cop/Lint/AmbiguousOperator.


@jamigibbs DWMR. Well done here - on this entire project. Not possible for an imposter. :+1:


There's one public behaviour change I notice - a ChildData from a TreeCache might now not be equal() to one from a PathChildrenCache where it was before. But that seems like a somewhat obscure scenario.
Looking at this gave me another idea though:
Instead of having a private subclass, PathChildrenCache could use the new ChildData directly, with the clearDataBytes() method changed to do a currentData.replace(data, new ChildData(data.getPath(), data.getStat(), null).
I think this would mean only a one-line change to PCC is needed and seems generally simpler. It would have the side effect of not clearing the data from ChildData objects that had been previously obtained from the cache - but I think this is probably a good thing too?
WDYT?


Flint would be proud of this change


Do you mind replacing "HEADS UP" with "WARNING"?

Otherwise this PR looks ready to merge.


modularizing


I agree with your arguments thus commit 1523531d3de7dc518cbaecf3f90b253ef0ec4cd4 reverts this one.


Ya you're correct, it skips a frame due to this change. Thanks for the note, i'll revert it!


Idk, maybe just typo


@shucknjiveturkey DWMR. Click click. Done done. Nice.


Oh for sure. I'll submit them tonight.
The only other thing to change (and if you want you can do so), is the
hint for the transmit button. Right now it says "tripple tap to
toggle", it should be "tripple-tap to toggle on, double-tap to toggle
off." With the redesign it somehow changed back, but the way to lock
/unlock it is the same when the setting for PTT lock is turned off.
Only a few more bugs / thoughts to raise:
When you get a phone call, you're disconnected from the server and it
has no microphone. (I've faced this one)
People have told me that when on iPhone speaker, the volume of
VoiceOver is significantly quieter than it used to be before the echo
cancelation was implemented.
Also, the Speaker output switch is also on iPads. Since iPads don't
have an earpiece by default, hiding it for iPads would make more
sense.
Otherwise, a work well done, Bjoern!
Sincerely,
Tamas
-Tamas

On 1/29/16, Bjoern D. Rasmussen <notifications@github.com> wrote:
> Yes, yes. Can you make them before sunday?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BearWare/TeamTalk5/commit/0b2b8035b679a504142349cf6462ca37c1827da3#commitcomment-15764276


Any idea what the performance impact/difference is?


nice one!


probably not a big advantage since the list is very small, but the dict lookup is fast, and the list has to be iterated to find the match


I feel something's wrong with this part (well, all parts on log4j.xml) of the commit. Okay, i get it that 'dockerize all the things' expects logging stuff to be output on the console/stdout of the process, but this removes all logging to a file which is probably useful to all the other non-docker deployments/users.

And i doubt the fact that things are logged on the console in parallel to a file is an issue with docker (maybe besides filesize growing indefinitely since there's no log rotation in a docker container?)

Either way, everyone probably customizes this file in the georchestra datadir so that's not a 'real issue', but the default-provided log4j.xml should still log to a file, as upstream intended...

cc @pmauduit @fvanderbiest


Ok, that's fine.


Fixed as well.


When doing what?


I tested with two backslashes.


pfffffffffffffff
So disappointing... Windows >> Linux !


@Aran85 need to replace "anonymous" with RGW_USER_ANON_ID


It's ok - even better - to call it once on `reloadWhenReady` instead of each `reload`


For [Docs issue#597](https://github.com/CartoDB/docs/issues/587)- WIP


I'm willing to change this if you feel it is better.


While trying to send a message from croogo default contact page.


The naming might be improper. The `peekCircuitBreaker` is used for ASQ operation using `GetMessages`. The `circuitBreaker` is used for actual action of pushing messages into pipeline. Should one of these be removed? Do you have any proposal for better names?


Thanks, again... :-)


Думаю, название шрифта должно выглядеть просто как `PT Sans`


Ok, thanks. I am going to add two backslashes.


To be honest I don't yet understand the difference between async and Promises well enough.

As implemented this still doesn't work in all cases.  If an action emits a thunk that emits an action and they run promises then the outer promise resolves before the inner promises and the count is off.

Although I did think the mock should have the same interface as the class it is mocking and the real AppAuthToken methods aren't declared async.


@dustyburwell Sorry I didn't get a chance to reply until now. I mostly agree with @jvoigtlaender. That's why both `elm-random-extra` and `elm-random-pcg` define `Random.constant` (akin to `succeed`) and `Random.andMap`.

You mentioned Haskell -- the problems you're running into are basically because Random doesn't have an equivalent of `pure`/`return` and `<*>`. Elm has been unwilling to standardize these operations the way Haskell has, and the current Random library is the result of that. I don't think that anyone would deny the importance or utility of `pure`, `fmap`, `<*>`, and `>>=`, it's just that we don't like those names, or that they're built on top of type classes.

If I may digress a bit further, we're still trying to figure out a good name for `pure`; `constant`, `succeed`, and `singleton` seem to each have their place. There's also a desire to flip the argument order for `andThen` to play better with `|>` but it hasn't happened. Finally, I wish we'd be more thorough about including `andMap` whenever there's `andThen`, for example in the `Signal` module.

Point being, you're not crazy, the API is incomplete! Why don't you try [Random.PCG](http://package.elm-lang.org/packages/mgold/elm-random-pcg/1.1.0/Random-PCG) which is a drop-in replacement for core's Random?


I don't mind :smile:
Just thinking of https://github.com/zikula/core/blob/1.4/src/docs/Core-2.0/Performance.md#usenamespacenotation


宕机上传不了


`CodeStyle: replace filehandler with the more mnemonic file `
->
`CodeStyle: filehandler -> file`


correct. nfetch() doesn't call is_404() at all--it just does the request. If you want to check for that condition you have to do it manually.


Awesome commit!


Thank you.


I've reread the description of `TransportTransactionMode.ReceiveOnly` and reread the part of the code responsible for that. I can see that `ReceiveResourceManager` is used to enlist in the current transaction. Changing this accordingly to your comment.


s/でもプログラム/デモプログラム/


LGTM


Not quite this simple, `/`, `/dev/` and `/dev/urandom` will all work.

Oh and it gets more fun, to quote [the manual](http://php.net/manual/en/ini.core.php#ini.open-basedir):

> The restriction specified with open_basedir is a directory name since PHP 5.2.16 and 5.3.4. Previous versions used it as a prefix. This means that "open_basedir = /dir/incl" also allowed access to "/dir/include" and "/dir/incls" if they exist. When you want to restrict access to only the specified directory, end with a slash. For example: open_basedir = /dir/incl/

Quick test I put together:

```sh
for path in / /d /de /dev /dev/ /dev/u /dev/urandom /dev/URANDOM; do printf '%s: ' "$path"; php -dopen_basedir="$path" -r 'var_dump(@is_readable("/dev/urandom"));'; done
/: bool(true)
/d: bool(false)
/de: bool(false)
/dev: bool(true)
/dev/: bool(true)
/dev/u: bool(false)
/dev/urandom: bool(true)
/dev/URANDOM: bool(false)
```


Aber, aber, das nutzt ja `unsafePerformIO` ... :cry:

```
scala> import scalaz._, Scalaz._, effect._
import scalaz._
import Scalaz._
import effect._

scala> object SafeHelloWorld extends SafeApp {
     |   override val runc = IO.putStrLn("Hello, World!")
     | }
defined object SafeHelloWorld

scala> SafeHelloWorld.main(Array())
Hello, World!
```


Salut,

Oui elles sont nécessaires. Nous avons décidé de les mettre en place afin de pouvoir gérer les sockets SSL. Elles sont censées être appelées au niveau du select. Cela implicite effectivement un buffer. Et si, les méthodes pour agir avec ce buffer sont les deux autres read & write. Nous n'avons pas explicitement déclaré ce buffer afin de laisser à chacun le choix dans son implémentation, mais c'est ainsi que fonctionne la plupart des API surcouchant les sockets, avec ces mêmes noms de fonction.

Nous ne suivons pas exactement les mêmes prototypes de C car ce sont loin d'être les meilleurs. Les notres nous semblent tout à fait utilisables.

Nous allons mettre à jour l'UML.

Bonne journée.


s/コマンドででも/コマンドで/


[R/exa.createScript.R:101:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L101) *style:* **Variable and function names should be all lowercase.**
```r
exa.createScript <- function(channel, name, func = NA,
^~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:103:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L103) *style:* **Variable and function names should be all lowercase.**
```r
                             initCode = NA,
                             ^~~~~~~~
```
[R/exa.createScript.R:104:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L104) *style:* **Variable and function names should be all lowercase.**
```r
                             cleanCode = NA,
                             ^~~~~~~~~
```
[R/exa.createScript.R:105:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L105) *style:* **Variable and function names should be all lowercase.**
```r
                             inType = SET,
                             ^~~~~~
```
[R/exa.createScript.R:106:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L106) *style:* **Variable and function names should be all lowercase.**
```r
                             inArgs = list(),
                             ^~~~~~
```
[R/exa.createScript.R:107:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L107) *style:* **Variable and function names should be all lowercase.**
```r
                             outType = EMITS,
                             ^~~~~~~
```
[R/exa.createScript.R:108:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L108) *style:* **Variable and function names should be all lowercase.**
```r
                             outArgs = list(),
                             ^~~~~~~
```
[R/exa.createScript.R:109:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L109) *style:* **Variable and function names should be all lowercase.**
```r
                             outputAddress = NA,
                             ^~~~~~~~~~~~~
```
[R/exa.createScript.R:110:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L110) *style:* **Variable and function names should be all lowercase.**
```r
                             replaceIfExists = TRUE,
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:111:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L111) *style:* **Variable and function names should be all lowercase.**
```r
                             mockOnly = FALSE
                             ^~~~~~~~
```
[R/exa.createScript.R:115:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
  ^~~~~~~~
```
[R/exa.createScript.R:115:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
                ^~~~~~~~
```
[R/exa.createScript.R:116:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
  ^~~~~~~~~
```
[R/exa.createScript.R:116:18:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
                 ^~~~~~~~~
```
[R/exa.createScript.R:118:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
  ^~~~~~
```
[R/exa.createScript.R:118:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                      ^~~~~~
```
[R/exa.createScript.R:118:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L118) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_IN_TYPES’**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                              ^~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:119:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
  ^~~~~~~
```
[R/exa.createScript.R:119:24:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                       ^~~~~~~
```
[R/exa.createScript.R:119:33:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L119) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_OUT_TYPES’**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                                ^~~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:121:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
  ^~~~~~
```
[R/exa.createScript.R:121:38:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
                                     ^~~~~~
```
[R/exa.createScript.R:123:7:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L123) *style:* **Variable and function names should be all lowercase.**
```r
  if (outType == EMITS) {
      ^~~~~~~
```
[R/exa.createScript.R:124:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L124) *style:* **Variable and function names should be all lowercase.**
```r
    if (is.null(m$outArgs)) {
                  ^~~~~~~
```
[R/exa.createScript.R:127:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L127) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- paste("(",
    ^~~~~~~
```
[R/exa.createScript.R:128:47:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L128) *style:* **Variable and function names should be all lowercase.**
```r
                     do.call(paste, c(as.list(outArgs), sep = ", ")),
                                              ^~~~~~~
```
[R/exa.createScript.R:131:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L131) *style:* **Variable and function names should be all lowercase.**
```r
    outType <- RETURNS
    ^~~~~~~
```
[R/exa.createScript.R:131:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L131) *warning:* **no visible binding for global variable ‘RETURNS’**
```r
    outType <- RETURNS
               ^~~~~~~
```
[R/exa.createScript.R:132:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
    ^~~~~~~
```
[R/exa.createScript.R:132:29:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
                            ^~~~~~~
```
[R/exa.createScript.R:135:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L135) *style:* **Variable and function names should be all lowercase.**
```r
  sql <- paste("CREATE", if (replaceIfExists) "OR REPLACE" else "", "R",
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:136:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L136) *style:* **Variable and function names should be all lowercase.**
```r
               inType, "SCRIPT", name,
               ^~~~~~
```
[R/exa.createScript.R:137:21:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L137) *style:* **Variable and function names should be all lowercase.**
```r
               "(", inArgs, ")",
                    ^~~~~~
```
[R/exa.createScript.R:138:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
               ^~~~~~~
```
[R/exa.createScript.R:138:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
                        ^~~~~~~
```
[R/exa.createScript.R:140:18:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L140) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(m$outputAddress))
                 ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:24:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                       ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:51:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                                                  ^~~~~~~~~~~~~
```
[R/exa.createScript.R:157:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L157) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(initCode)) {
               ^~~~~~~~
```
[R/exa.createScript.R:159:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:159:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
                             ^~~~~~~~
```
[R/exa.createScript.R:160:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L160) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:165:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L165) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(cleanCode)) {
               ^~~~~~~~~
```
[R/exa.createScript.R:167:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:167:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
                             ^~~~~~~~~
```
[R/exa.createScript.R:168:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L168) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:173:8:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L173) *style:* **Variable and function names should be all lowercase.**
```r
  for (codeLine in deparse(code)) {
       ^~~~~~~~
```
[R/exa.createScript.R:174:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L174) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste(sql, codeLine, sep = "\n")
                      ^~~~~~~~
```
[R/exa.createScript.R:178:8:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L178) *style:* **Variable and function names should be all lowercase.**
```r
  if (!mockOnly) {
       ^~~~~~~~
```
[R/exa.createScript.R:179:9:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L179) *style:* **Variable and function names should be all lowercase.**
```r
    if (odbcQuery(channel, sql) == -1) {
        ^~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L180) *style:* **Variable and function names should be all lowercase.**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L180) *warning:* **no visible global function definition for ‘odbcGetErrMsg’**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:185:36:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                   ^~~~~~~
```
[R/exa.createScript.R:185:50:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                                 ^~~~~~~~~
```
[R/exa.createScript.R:186:12:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L186) *style:* **Variable and function names should be all lowercase.**
```r
           returnSQL = FALSE, reader = NA, server = NA) {
           ^~~~~~~~~
```
[R/exa.createScript.R:189:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L189) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(table))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:190:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L190) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:192:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L192) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(where))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:193:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L193) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L195) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(groupBy))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L195) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(groupBy))
                             ^~~~~~~
```
[R/exa.createScript.R:196:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L196) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(groupBy) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:196:24:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L196) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(groupBy) == 1)
                       ^~~~~~~
```
[R/exa.createScript.R:198:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L198) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(restQuery))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:198:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L198) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(restQuery))
                             ^~~~~~~~~
```
[R/exa.createScript.R:199:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L199) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(restQuery) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:199:24:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L199) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(restQuery) == 1)
                       ^~~~~~~~~
```
[R/exa.createScript.R:213:9:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L213) *style:* **Variable and function names should be all lowercase.**
```r
    if (groupBy != "") {
        ^~~~~~~
```
[R/exa.createScript.R:214:7:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
      ^~~~~~~
```
[R/exa.createScript.R:214:37:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
                                    ^~~~~~~
```
[R/exa.createScript.R:218:32:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L218) *style:* **Variable and function names should be all lowercase.**
```r
                 table, where, groupBy, sep = "")
                               ^~~~~~~
```
[R/exa.createScript.R:219:47:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L219) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste("SELECT * FROM (", sql, ")", restQuery, sep = "")
                                              ^~~~~~~~~
```
[R/exa.createScript.R:221:9:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L221) *style:* **Variable and function names should be all lowercase.**
```r
    if (returnSQL) {
        ^~~~~~~~~
```
[R/exa.createScript.R:224:7:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L224) *style:* **Variable and function names should be all lowercase.**
```r
      execArgs <- list(channel, sql)
      ^~~~~~~~
```
[R/exa.createScript.R:226:9:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
        ^~~~~~~~
```
[R/exa.createScript.R:226:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
                      ^~~~~~~~
```
[R/exa.createScript.R:229:9:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
        ^~~~~~~~
```
[R/exa.createScript.R:229:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
                      ^~~~~~~~
```
[R/exa.createScript.R:231:12:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L231) *style:* **Variable and function names should be all lowercase.**
```r
      if (!mockOnly) {
           ^~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L232) *warning:* **no visible binding for global variable ‘exa.readData’**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                              ^~~~~~~~
```
[R/EXADBI-internal.R:2:2:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L2) *style:* **Commented code should be removed.**
```r
## These are sub-methods used by the DBI methods defined in EXADBI.R.
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:4:2:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L4) *style:* **Commented code should be removed.**
```r
## First version written in 2015 by Marcel Boldt <marcel.boldt@exasol.com>
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:5:2:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L5) *style:* **Commented code should be removed.**
```r
## as part of the EXASOL R interface & SDK package. It may be used, changed and distributed freely
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:15:30:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L15) *style:* **Commas should never have a space before.**
```r
grep_schema <- function(stmt , statement = TRUE) {
                            ~^
```
[R/EXADBI-internal.R:16:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L16) *style:* **lines should not be more than 80 characters.**
```r
  g1 <-   gregexpr("from(([\\s.]\"[^\"]+\")|([\\s.]\\w+))+?([;\\s)]|$)",  # grep all from stmts.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:18:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L18) *style:* **lines should not be more than 80 characters.**
```r
  ids <- gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:18:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L18) *style:* **Commas should always have a space after.**
```r
  ids <- gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
                      ^
```
[R/EXADBI-internal.R:18:26:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L18) *style:* **Commas should always have a space after.**
```r
  ids <- gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
                         ^
```
[R/EXADBI-internal.R:23:27:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L23) *style:* **Commented code should be removed.**
```r
  #                       return(substr(x,1,nchar(x)-1))
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:30:29:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L30) *style:* **Commas should always have a space after.**
```r
  schemas <- substr(schemas,1,nchar(schemas)-1)
                            ^
```
[R/EXADBI-internal.R:30:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L30) *style:* **Commas should always have a space after.**
```r
  schemas <- substr(schemas,1,nchar(schemas)-1)
                              ^
```
[R/EXADBI-internal.R:30:45:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L30) *style:* **Put spaces around all infix operators.**
```r
  schemas <- substr(schemas,1,nchar(schemas)-1)
                                           ~^~
```
[R/EXADBI-internal.R:43:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L43) *style:* **lines should not be more than 80 characters.**
```r
        substr(identifier[i], nchar(identifier[i]), nchar(identifier[i])) == "\"") {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:61:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L61) *style:* **Variable and function names should be all lowercase.**
```r
processIDs <- function(id, quotes = "\"") {
^~~~~~~~~~
```
[R/EXADBI-internal.R:63:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L63) *style:* **Put spaces around all infix operators.**
```r
  if (length(id <=1)) if(is.na(id) | is.null(id) | missing(id)) return(id)
                ^~~
```
[R/EXADBI-internal.R:63:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L63) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if (length(id <=1)) if(is.na(id) | is.null(id) | missing(id)) return(id)
                        ^
```
[R/EXADBI-internal.R:67:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L67) *style:* **Commas should always have a space after.**
```r
      if (substr(x, 1,1) == "\"")
                      ^
```
[R/EXADBI-internal.R:81:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L81) *style:* **Only use double-quotes.**
```r
    gsub('.', '_', id, fixed = TRUE)# change dots to underscores
         ^~~
```
[R/EXADBI-internal.R:81:15:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L81) *style:* **Only use double-quotes.**
```r
    gsub('.', '_', id, fixed = TRUE)# change dots to underscores
              ^~~
```
[R/EXADBI-internal.R:83:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L83) *style:* **lines should not be more than 80 characters.**
```r
    paste('"', encodeString(id), '"', sep = "") # escape non-printable chr & add surrounding quotes
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:99:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L99) *style:* **Variable and function names should be all lowercase.**
```r
EXAGetIdentifier <-
^~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:107:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L107) *style:* **lines should not be more than 80 characters.**
```r
          "from(([\\s.]\"[^\"]+\")|([\\s.]\\w+))+?([;\\s)]|$)",  # grep all from stmts.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:111:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L111) *style:* **lines should not be more than 80 characters.**
```r
        gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:111:22:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L111) *style:* **Commas should always have a space after.**
```r
        gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
                     ^
```
[R/EXADBI-internal.R:111:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L111) *style:* **Commas should always have a space after.**
```r
        gsub("from ","",unlist(regmatches(stmt, g1)), ignore.case = TRUE) # remove the "from "s
                        ^
```
[R/EXADBI-internal.R:116:49:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L116) *style:* **Commas should always have a space after.**
```r
          if (substring(x, nchar(x)) %in% c(";"," ",")"))
                                                ^
```
[R/EXADBI-internal.R:116:53:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L116) *style:* **Commas should always have a space after.**
```r
          if (substring(x, nchar(x)) %in% c(";"," ",")"))
                                                    ^
```
[R/EXADBI-internal.R:117:11:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L117) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
          {
          ^
```
[R/EXADBI-internal.R:118:29:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L118) *style:* **Commas should always have a space after.**
```r
            return(substr(x,1,nchar(x) - 1))
                            ^
```
[R/EXADBI-internal.R:118:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L118) *style:* **Commas should always have a space after.**
```r
            return(substr(x,1,nchar(x) - 1))
                              ^
```
[R/EXADBI-internal.R:132:24:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L132) *style:* **Commas should always have a space after.**
```r
        substr(schemas,1,nchar(schemas) - 1) # remove the final dot
                       ^
```
[R/EXADBI-internal.R:132:26:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L132) *style:* **Commas should always have a space after.**
```r
        substr(schemas,1,nchar(schemas) - 1) # remove the final dot
                         ^
```
[R/EXADBI-internal.R:136:20:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L136) *style:* **Variable and function names should be all lowercase.**
```r
        schemas <- processIDs(schemas, quotes = quotes)
                   ^~~~~~~~~~
```
[R/EXADBI-internal.R:136:20:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L136) *warning:* **no visible global function definition for ‘processIDs’**
```r
        schemas <- processIDs(schemas, quotes = quotes)
                   ^~~~~~~~~~
```
[R/EXADBI-internal.R:138:20:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L138) *warning:* **no visible global function definition for ‘EXAupper’**
```r
        schemas <- EXAupper(schemas)
                   ^~~~~~~~
```
[R/EXADBI-internal.R:148:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L148) *style:* **Variable and function names should be all lowercase.**
```r
        tables <- processIDs(tables, quotes = quotes)
                  ^~~~~~~~~~
```
[R/EXADBI-internal.R:148:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L148) *warning:* **no visible global function definition for ‘processIDs’**
```r
        tables <- processIDs(tables, quotes = quotes)
                  ^~~~~~~~~~
```
[R/EXADBI-internal.R:150:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L150) *warning:* **no visible global function definition for ‘EXAupper’**
```r
        tables <- EXAupper(tables)
                  ^~~~~~~~
```
[R/EXADBI-internal.R:157:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L157) *style:* **lines should not be more than 80 characters.**
```r
    # as.data.frame(ds, stringsAsFactors = FALSE, optional = TRUE) # doesn't work as it strips quotes
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:157:7:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L157) *style:* **Commented code should be removed.**
```r
    # as.data.frame(ds, stringsAsFactors = FALSE, optional = TRUE) # doesn't work as it strips quotes
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:159:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L159) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/EXADBI-internal.R:160:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L160) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/EXADBI-internal.R:161:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI-internal.R#L161) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/EXADBI.R:10:2:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L10) *style:* **Commented code should be removed.**
```r
## First version written in 2015 by Marcel Boldt <marcel.boldt@exasol.com>
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:11:2:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L11) *style:* **Commented code should be removed.**
```r
## as part of the EXASOL R interface & SDK package. It may be used, changed and distributed freely
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:64:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L64) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:89:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L89) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:103:44:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L103) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:153:15:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L153) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAConnection",
              ^
```
[R/EXADBI.R:154:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L154) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:155:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L155) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
         ^~~~~~~~~
```
[R/EXADBI.R:155:20:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L155) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
                   ^~~~~
```
[R/EXADBI.R:159:26:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                         ^~~~~
```
[R/EXADBI.R:159:46:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                                             ^~~~~
```
[R/EXADBI.R:160:16:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L160) *style:* **Variable and function names should be all lowercase.**
```r
      dbname = dbObj@db_name,
               ^~~~~
```
[R/EXADBI.R:161:18:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L161) *style:* **Variable and function names should be all lowercase.**
```r
      username = dbObj@db_user,
                 ^~~~~
```
[R/EXADBI.R:162:14:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L162) *style:* **Variable and function names should be all lowercase.**
```r
      host = dbObj@db_host,
             ^~~~~
```
[R/EXADBI.R:163:14:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L163) *style:* **Variable and function names should be all lowercase.**
```r
      port = dbObj@db_port
             ^~~~~
```
[R/EXADBI.R:172:44:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L172) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:223:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L223) *style:* **lines should not be more than 80 characters.**
```r
    #         close = function(commit=TRUE) {                              # dbClearResult is in row 880
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:224:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L224) *style:* **lines should not be more than 80 characters.**
```r
    #           "Frees up all resources, in particular drops the temporary table in the DB."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:227:21:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L227) *style:* **Commented code should be removed.**
```r
    #               warning("Connection seems exipired.\n    ...it's gone...")
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:228:21:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L228) *style:* **Commented code should be removed.**
```r
    #               return(FALSE)
                    ^~~~~~~~~~~~~
```
[R/EXADBI.R:232:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L232) *style:* **Commented code should be removed.**
```r
    #                 if(commit) message("Changes commited.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:233:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L233) *style:* **lines should not be more than 80 characters.**
```r
    #             } else stop("Commit failed. Changes NOT commited. Closing aborted.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:234:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L234) *style:* **Commented code should be removed.**
```r
    #           message("Closing connection...")
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:235:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L235) *style:* **Commented code should be removed.**
```r
    #             res <- try(odbcClose(connection),silent=TRUE)
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:237:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L237) *style:* **Commented code should be removed.**
```r
    #                 message("Connection closed.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:238:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L238) *style:* **Commented code should be removed.**
```r
    #                 return(TRUE)
                      ^~~~~~~~~~~~
```
[R/EXADBI.R:240:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L240) *style:* **Commented code should be removed.**
```r
    #                 warning("Closing not successful.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:241:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L241) *style:* **Commented code should be removed.**
```r
    #                 return(FALSE)
                      ^~~~~~~~~~~~~
```
[R/EXADBI.R:243:27:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L243) *style:* **Commented code should be removed.**
```r
    #                     warning(res)
                          ^~~~~~~~~~~~
```
[R/EXADBI.R:244:27:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L244) *style:* **Commented code should be removed.**
```r
    #                     return(FALSE)
                          ^~~~~~~~~~~~~
```
[R/EXADBI.R:249:8:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L249) *style:* **Commented code should be removed.**
```r
      #close()
       ^~~~~~~
```
[R/EXADBI.R:256:15:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L256) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAResult",
              ^
```
[R/EXADBI.R:257:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L257) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:259:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L259) *style:* **Variable and function names should be all lowercase.**
```r
      statement = dbObj$statement,
                  ^~~~~
```
[R/EXADBI.R:260:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L260) *style:* **Variable and function names should be all lowercase.**
```r
      row.count = dbObj$rows_fetched,
                  ^~~~~
```
[R/EXADBI.R:261:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L261) *style:* **Variable and function names should be all lowercase.**
```r
      rows.affected = dbObj$rows_affected,
                      ^~~~~
```
[R/EXADBI.R:262:23:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L262) *style:* **Variable and function names should be all lowercase.**
```r
      has.completed = dbObj$is_complete,
                      ^~~~~
```
[R/EXADBI.R:263:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L263) *style:* **Variable and function names should be all lowercase.**
```r
      is.select = dbObj$with_output
                  ^~~~~
```
[R/EXADBI.R:271:44:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L271) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:281:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L281) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:286:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L286) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:296:25:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L296) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:307:47:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L307) *style:* **Variable and function names should be all lowercase.**
```r
setMethod("dbDataType", "EXAObject", function(dbObj, obj, ...) {
                                              ^~~~~
```
[R/EXADBI.R:308:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L308) *style:* **Variable and function names should be all lowercase.**
```r
  EXADataType(obj)
  ^~~~~~~~~~~
```
[R/EXADBI.R:313:13:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L313) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:331:13:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L331) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:338:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L338) *style:* **Variable and function names should be all lowercase.**
```r
    dbGetInfo(drv, "connectionIds")[[1]]
    ^~~~~~~~~
```
[R/EXADBI.R:400:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L400) *style:* **lines should not be more than 80 characters.**
```r
                        exalogfile = tempfile(pattern = "EXAODBC_", fileext = ".log"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:405:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L405) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcctype = Sys.getlocale(category = "LC_CTYPE"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:406:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L406) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcnumeric = Sys.getlocale(category = "LC_NUMERIC"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:410:3:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L410) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {
  ^
```
[R/EXADBI.R:411:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L411) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:440:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L440) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:440:28:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L440) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
                           ^~~~~~~~
```
[R/EXADBI.R:451:5:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L451) *style:* **Variable and function names should be all lowercase.**
```r
    EXACloneConnection(drv, ...),
    ^~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:463:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L463) *style:* **Variable and function names should be all lowercase.**
```r
dbCurrentSchema <- function(con) {
^~~~~~~~~~~~~~~
```
[R/EXADBI.R:464:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L464) *style:* **Variable and function names should be all lowercase.**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:464:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L464) *warning:* **no visible global function definition for ‘sqlQuery’**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:465:44:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L465) *style:* **Commas should always have a space after.**
```r
  con@current_schema <- as.character(res[1,1])
                                           ^
```
[R/EXADBI.R:472:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L472) *style:* **Variable and function names should be all lowercase.**
```r
EXANewConnection <- function(# change defaults also above
^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:495:32:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L495) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0("DSN=",dsn)
                               ^
```
[R/EXADBI.R:498:64:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L498) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0("DRIVER={EXASolution Driver};EXAHOST=",exahost)
                                                               ^
```
[R/EXADBI.R:502:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L502) *style:* **lines should not be more than 80 characters.**
```r
        "Connect failed. Either DSN, host & db_user or a connection string must be given.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:503:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L503) *style:* **lines should not be more than 80 characters.**
```r
        Hint: No lazy declaration of onnection parameters - these have to be stated ' dsn=...'.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:509:33:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L509) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                ^
```
[R/EXADBI.R:509:41:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L509) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                        ^
```
[R/EXADBI.R:509:45:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L509) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                            ^
```
[R/EXADBI.R:509:53:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L509) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                                    ^
```
[R/EXADBI.R:513:33:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L513) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                ^
```
[R/EXADBI.R:513:47:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L513) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                              ^
```
[R/EXADBI.R:516:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                              ^
```
[R/EXADBI.R:516:46:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                                             ^
```
[R/EXADBI.R:519:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L519) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";LOGMODE=",logmode)
                              ^
```
[R/EXADBI.R:519:43:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L519) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";LOGMODE=",logmode)
                                          ^
```
[R/EXADBI.R:524:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L524) *style:* **lines should not be more than 80 characters.**
```r
        con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:524:17:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
        con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric
                ^
```
[R/EXADBI.R:524:39:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
        con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric
                                      ^
```
[R/EXADBI.R:524:57:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
        con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric
                                                        ^
```
[R/EXADBI.R:524:81:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
        con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric
                                                                                ^
```
[R/EXADBI.R:529:31:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L529) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";autocommit=",autocommit)
                              ^
```
[R/EXADBI.R:529:46:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L529) *style:* **Commas should always have a space after.**
```r
    con_str <- paste0(con_str,";autocommit=",autocommit)
                                             ^
```
[R/EXADBI.R:534:33:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L534) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                ^
```
[R/EXADBI.R:534:50:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L534) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                 ^
```
[R/EXADBI.R:534:54:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L534) *style:* **Commas should always have a space after.**
```r
      con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                     ^
```
[R/EXADBI.R:539:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L539) *style:* **Variable and function names should be all lowercase.**
```r
  con <- odbcDriverConnect(con_str)
         ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:539:10:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L539) *warning:* **no visible global function definition for ‘odbcDriverConnect’**
```r
  con <- odbcDriverConnect(con_str)
         ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:540:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L540) *style:* **Variable and function names should be all lowercase.**
```r
  exa_metadata <- odbcGetInfo(con)
                  ^~~~~~~~~~~
```
[R/EXADBI.R:540:19:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L540) *warning:* **no visible global function definition for ‘odbcGetInfo’**
```r
  exa_metadata <- odbcGetInfo(con)
                  ^~~~~~~~~~~
```
[R/EXADBI.R:543:21:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L543) *style:* **Commas should always have a space after.**
```r
    "EXAConnection",init_connection_string = con_str,
                    ^
```
[R/EXADBI.R:545:48:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L545) *style:* **Commas should always have a space after.**
```r
    autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                               ^
```
[R/EXADBI.R:545:53:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L545) *style:* **Commas should always have a space after.**
```r
    autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                                    ^
```
[R/EXADBI.R:546:52:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L546) *style:* **Commas should always have a space after.**
```r
    db_host = strsplit(exa_metadata["Server_Name"],":")[[1]][1],
                                                   ^
```
[R/EXADBI.R:547:63:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L547) *style:* **Commas should always have a space after.**
```r
    db_port = as.numeric(strsplit(exa_metadata["Server_Name"],":")[[1]][2]),
                                                              ^
```
[R/EXADBI.R:550:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L550) *style:* **lines should not be more than 80 characters.**
```r
        attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl =
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:550:103:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L550) *style:* **Commas should always have a space after.**
```r
        attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl =
                                                                                ^
```
[R/EXADBI.R:555:1:](https://github.com/marcelboldt/r-exasol/blob/51438640b1cf32a5262c4ae12ea585b515e2e62b/R/EXADBI.R#L555) *style:* **lines should not be more than 80 characters.**
```r
        attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl =
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Looks like "canola" is misspelled!


Prevents segfaults when expected file size is too small. Always a good thing, avoiding segfaults.


:smirk: