> Because I'm assuming someone might want to change the default command in the future. We want to keep the devtools ones somewhere for users who want it, with an easy to use map.

This is not a good reason for proliferation of small commands that do almost same trick. Let's aim at the same style generics as we do elsewhere. In the future we will drop the -r- infix and will make them generic with `ess-defgeneric`. If users want extra devtools commands they can copy paste the code and make their own. Or just customize the command string.


Bien en general, como detalle (por mencionar algo) mirar el code style guide https://github.com/globant-ui/JavaScript-style-guide


Nope, I will fix it


It does not look good.


Perfect explanation


Over four years now.. :smiley:


>Neglecting this is a big mistake.
I absolutely agree. Bytw with the INDI module, PixInight would be the
_only_ platform (afaik) which supports INDI on all relevant platform
(windows,linux,os).

2016-04-29 14:05 GMT+02:00 Juan Conejero <notifications@github.com>:

> actually these changes are even not commited to the INDI repo ... maybe
> because they dont like windows ;)...
>
> The lack of robust Windows support is INDI project's main problem, IMO,
> and it is a pity. The ASCOM "platform" is Microsoft-centric, unportable,
> and wrongly conceived and designed. One of INDI's main goals *should* be
> replacing that old beast completely in the medium-long term. I also don't
> like Windows at all, but this fact does not prevent me from developing a
> Windows version of PixInsight. It is just my job, and I have to get it done
> if I want to survive, that's how things are. Neglecting this is a big
> mistake.
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/PixInsight/PCL/commit/802889cca816e265d3aa28683eeabb48d42899be#commitcomment-17302245>
>


Thanks


Yeah, its a bit of a mess.
I'll change it so you have to configure where R.exe / R lives.
Then at startup we'll run it: R RHOME

If it runs _and_ we get a valid directory we can start, if not no dice.

I'll also add a --noR command line flag.


these file is too big. so divded one file into two. so Please add two file.


Code need some description.


 >these little problems (well, or not so little ;-)
It think little ;) ..

2016-04-29 13:53 GMT+02:00 Juan Conejero <notifications@github.com>:

> Hi Klaus,
>
> In the next release we should cleanup the INDI baseclient code, or write
> our own client implementation.
>
> Yeah, that code is ugly as sin, for sure. In a first stage we might try to
> clean it as possible, but I agree that we should write our own client base
> routines. Let's release a first version and stabilize it now, but this is
> definitely in the short-term to-do list.
>
> As for the issues on Windows, take your time to fix them. The module is
> essentially ready for release, so as soon as you find and solve these
> little problems (well, or not so little ;-) ), we'll be ready for prime
> time.
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/PixInsight/PCL/commit/06cdd542359ad5ef35963bc2b594062212b1917f#commitcomment-17302126>
>


卧槽


29 апреля 2016 г., 15:31 пользователь Anton Rodin <notifications@github.com>
написал:

> я его эксплуатирую (=

))) гениально, а ты ещё не пытался подписывать xml запросы?

--
С Уважением Михаил


@Setnea, did you attach some tests that check map_plate module ?


Good afternoon.
The program says that there is a new update, but the process is interrupted.
When will be able to upgrade?


Why is it a pointer to boolean, not just a boolean value?


Get from XML


Don't copy-paste constant another module is responsible for.
You can always get current dev size from vsd_dev->hwregs->dev_size.
So there is no even need for copy-pasting.


Faudrait changer le setOrient pour qu'il mette les "Left", "Right", etc dynamiquement sinon ça va faire beaucoup de code inutile si on rajoute d'autres créatures.
Faut il a changé les import de Window et Map tout seul, faudra les refaire parce que Map devrait pas connaitre Game


sleep value as parameter in XML settings


Fix here https://github.com/galaxyproject/galaxy/pull/2271 - I tested with both username and email as identifier. The new integration test framework would make it really easy to test these configuration parameters - I'll write a test next time I touch upload.


It's done


Yep. Very good.


ah cool, thanks! i'll make use of that


> This is not a good reason for proliferation of small commands that do almost same trick.

I think it is. And there's also the reason of including all the little features of devtools such as modification of the DESCRIPTION file that I'd like to include. It just seems cleaner to have a `devtools-` namespace for those.

> will make them generic with ess-defgeneric

Exactly. And then we can choose to refer to `ess-r-devtools-xxx` or `ess-r-pacman-yyy` or whatever on a case by case basis. How could it be cleaner than that? It's not like it would be a great maintaining burden to keep those commands.


I'll include this change in the qgis Debian package to resolve [Debian Bug #822477](https://bugs.debian.org/822477), it's unfortunate that this change didn't make it into todays 2.14.2 release. I look forward to dropping the patch for update to 2.14.3.


Should not the FLAG_crankshaft returned to previous value not to hard-coded "true" constant?


Hi, original author of the patch here. :)

I'm curious: were these patches reverted because you experienced problems with them? If so, I'd be very interested to know what the problems were, so I can try to reproduce and fix them!

Thanks in advance.


Should not you store previous value of FLAG_crankshaft?


Sorry I forgot about this, did @k21's comments in #717 clear things up?


total. hack.


You shouldn't copy-paste constants that you module is not responsible for.
Also 2 * PAGE_SIZE is not constant because it may be changed during vsd_device module load module_param(buf_size, ulong, S_IRUGO);
You can easily get current vsd device size using vsd_dev->hwregs->dev_size.
So... 3 reasons not to do it like that :))


TeamCity Mobile :: Android SDK (develop) [Build 434](http://teamcity.hq.judo/viewLog.html?buildId=26594&buildTypeId=JudoAndroid_AndroidGradle) is now running


So I don't see any write to free kernel buffer here. So you answer is not accepted. It is not precise at least.


idiot


No mutex_lock(&vsd_dev->dev_ops_serialization_mutex);
Don't hurry up when you code in kernel :)


We have to threw out all xml settings from git repostiory


"more user friendly"


Code duplication #18


what about 8dcd918?


You shouldn't copy-paste constants that you module is not responsible for.
Also 2 * PAGE_SIZE is not constant because it may be changed during vsd_device module load module_param(buf_size, ulong, S_IRUGO);
You can easily get current vsd device size using vsd_dev->hwregs->dev_size.
So... 3 reasons not to do it like that.


Quick (possibly random, unrelated) question:

Do you know how to get slurping and barfing Clojure s-expressions inside spacemacs?

I'm trying to use [evil-paredit](https://github.com/sw00/dotfiles/blob/master/spacemacs#L44) but [keybindings](https://github.com/sw00/dotfiles/blob/master/spacemacs#L256) not working.

Maybe you have some inside knowledge and arcane emacs trickery?


No, I didn't have a chance to test this yet. I'm hoping to get it done this weekend.


More or less right.
When not interrupt but signal is delivered to task waiting on wait_queue it is woken up.
Then this task frees the buffer. And then "When device getting control, it write to buffer".
I accept this answer.


Does this only apply to Mac? At least I didn't see `shell command` in linux version code 1.0.0.


actually gonna remove this one as it sets the cooldown for all colors of signal flare, which is meh


TeamCity Mobile :: Android SDK (develop) [Build 434](http://teamcity.hq.judo/viewLog.html?buildId=26594&buildTypeId=JudoAndroid_AndroidGradle) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 105, ignored: 1 Build time: 00:05:16
### Failed tests
```
com.judopay.ActivityResultTest: com.judopay.payment.DeclinedPaymentTest.shouldDeclineInvalidMaestroWhenAvsEnabled: <no details avaliable>

```


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.728.7caf168ab421810c69a5ce943d1b7ae6c217df51](http://build.palaso.org/viewLog.html?buildId=71670&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3469, ignored: 126 Build time: 00:06:59


What are the libraries that have to create for Apk works
Do you have any specific name? Or just use the ndk


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#cJzfLf_STI6WjrnIgmCPNg


PAC的代理端口成了8086，http://127.0.0.1:8086/proxy.pac


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.736.7caf168ab421810c69a5ce943d1b7ae6c217df51](http://build.palaso.org/viewLog.html?buildId=71671&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3449, ignored: 107 Build time: 00:06:49


You've forgotten to synchronize on dev_ops_serialization_mutex in every vsd_driver syscall


Got it! IT's working now, thanks for the help.


Então, olhá lá: https://github.com/golgota/librioteca.

É umas badges que ficam atualizadas com o estado atual do projeto no code climate, quando tivermos o semaphore vamos botar uma para ele e a de coverage dos testes vai ficar ativa.


汗，这。。。


erre kell visszaállítani valahogy és jólesz


thanks


EINVAL


新加一个 commit, r441 应该改回来了。


cleanups_and_reworkds (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6: OK (394 of 399 tests passed, 0 test warnings)

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


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


cleanups_and_reworkds (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-LibC++: Build Failed

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


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#LPpU3zRxR62ltRfjSz73KQ


Shouldn't this have been written like the below (as generated by paster / mr.bob etc)?

    entry_points="""
          [z3c.autoinclude.plugin]
          target = plone
          """,
    ...


Hi! Why didn't you do something like
```cs
Task.WaitAll(_partitionPollingIndex.Values.Where(task => task != null)).Wait(TimeSpan.FromSeconds(5))
```
this?


После праздников буду просить чела, чтобы тот сдела COM-обьект для подписи, а пока еще ни чего не смог подписать


> appendToは実行されるのでしょうか？

動きますよー。

var は「変数を定義する」という意味なのは正しいです。その初期値を`newSpan.appendTo(newDiv);` で入れています。appendToを実行した戻り値（ここではappendToで作成された要素を指す新しいjQueryオブジェクト）で初期化されます。

> varの定義を下のクリックイベントに引き継ぐということですよね？

残念、逆ですよ。 27行目でやっている処理は`button.next` に対してイベントをくっつけていますよね？その`button.next`というのはこの部分の処理で作られたものですよ。 $span から`find`すれば`button.next`を探すこと、できませんか？


the hack documentation says it can be used several times. Is it expected to implement a different behavior here ?


@mannay コメント返信しましたが、どうですか？

コメントへの返信は24時間（最大48時間）程度は間が空くと思っててくだされ。


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#PLr2E5-qRq6r2nlHFG8_Sw


The result of (BoaMathIntrinsics.random.nextLong() % (n/2) + (n/2)) is not uniform distribution.


[R/exa.createScript.R:103:30:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:119:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:121:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.createScript.R#L131) *style:* **Variable and function names should be all lowercase.**
```r
    outType <- RETURNS
    ^~~~~~~
```
[R/exa.createScript.R:132:5:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.createScript.R#L180) *style:* **Variable and function names should be all lowercase.**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:185:36:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:195:30:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:24:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:30:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:24:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.createScript.R#L231) *style:* **Variable and function names should be all lowercase.**
```r
      if (!mockOnly) {
           ^~~~~~~~
```
[R/exa.createScript.R:232:31:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                              ^~~~~~~~
```
[R/exa.readData.R:48:53:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L48) *style:* **Only use double-quotes.**
```r
exa.readData <- function(channel, query, encoding = 'UTF-8',
                                                    ^~~~~~~
```
[R/exa.readData.R:49:46:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L49) *style:* **Commas should always have a space after.**
```r
                         reader = function(x,..., enc = encoding) {
                                             ^
```
[R/exa.readData.R:50:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L50) *style:* **lines should not be more than 80 characters.**
```r
                           read.csv(x,..., stringsAsFactors = FALSE, encoding = enc,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.readData.R:50:39:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L50) *style:* **Commas should always have a space after.**
```r
                           read.csv(x,..., stringsAsFactors = FALSE, encoding = enc,
                                      ^
```
[R/exa.readData.R:51:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L51) *style:* **lines should not be more than 80 characters.**
```r
                                    blank.lines.skip = FALSE, numerals="no.loss")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.readData.R:51:71:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L51) *style:* **Put spaces around all infix operators.**
```r
                                    blank.lines.skip = FALSE, numerals="no.loss")
                                                                     ~^~
```
[R/exa.readData.R:53:38:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L53) *style:* **Commas should always have a space after.**
```r
                         server = NA,...) {
                                     ^
```
[R/exa.readData.R:60:15:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L60) *style:* **Variable and function names should be all lowercase.**
```r
    server <- odbcGetInfo(channel)[["Server_Name"]]
              ^~~~~~~~~~~
```
[R/exa.readData.R:63:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  serverAddress <- strsplit(server, ":")[[1]]
  ^~~~~~~~~~~~~
```
[R/exa.readData.R:65:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L65) *style:* **Variable and function names should be all lowercase.**
```r
  serverHost <- as.character(serverAddress[[1]])
  ^~~~~~~~~~
```
[R/exa.readData.R:65:30:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L65) *style:* **Variable and function names should be all lowercase.**
```r
  serverHost <- as.character(serverAddress[[1]])
                             ^~~~~~~~~~~~~
```
[R/exa.readData.R:66:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L66) *style:* **Variable and function names should be all lowercase.**
```r
  serverPort <- as.integer(serverAddress[[2]])
  ^~~~~~~~~~
```
[R/exa.readData.R:66:28:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L66) *style:* **Variable and function names should be all lowercase.**
```r
  serverPort <- as.integer(serverAddress[[2]])
                           ^~~~~~~~~~~~~
```
[R/exa.readData.R:68:36:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L68) *style:* **Variable and function names should be all lowercase.**
```r
  .Call(C_asyncRODBCIOStart, slot, serverHost, serverPort)
                                   ^~~~~~~~~~
```
[R/exa.readData.R:68:48:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L68) *style:* **Variable and function names should be all lowercase.**
```r
  .Call(C_asyncRODBCIOStart, slot, serverHost, serverPort)
                                               ^~~~~~~~~~
```
[R/exa.readData.R:70:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L70) *style:* **Variable and function names should be all lowercase.**
```r
  proxyHost <- .Call(C_asyncRODBCProxyHost, slot)
  ^~~~~~~~~
```
[R/exa.readData.R:71:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L71) *style:* **Variable and function names should be all lowercase.**
```r
  proxyPort <- .Call(C_asyncRODBCProxyPort, slot)
  ^~~~~~~~~
```
[R/exa.readData.R:72:64:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L72) *style:* **Variable and function names should be all lowercase.**
```r
  query <- paste("EXPORT (", query, ") INTO CSV AT 'http://",  proxyHost, ":",
                                                               ^~~~~~~~~
```
[R/exa.readData.R:73:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L73) *style:* **lines should not be more than 80 characters.**
```r
                 proxyPort, "' FILE 'executeSQL.csv' ENCODING = '",encoding,"' BOOLEAN = 'TRUE/FALSE' WITH COLUMN NAMES",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.readData.R:73:18:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L73) *style:* **Variable and function names should be all lowercase.**
```r
                 proxyPort, "' FILE 'executeSQL.csv' ENCODING = '",encoding,"' BOOLEAN = 'TRUE/FALSE' WITH COLUMN NAMES",
                 ^~~~~~~~~
```
[R/exa.readData.R:73:68:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L73) *style:* **Commas should always have a space after.**
```r
                 proxyPort, "' FILE 'executeSQL.csv' ENCODING = '",encoding,"' BOOLEAN = 'TRUE/FALSE' WITH COLUMN NAMES",
                                                                   ^
```
[R/exa.readData.R:73:77:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L73) *style:* **Commas should always have a space after.**
```r
                 proxyPort, "' FILE 'executeSQL.csv' ENCODING = '",encoding,"' BOOLEAN = 'TRUE/FALSE' WITH COLUMN NAMES",
                                                                            ^
```
[R/exa.readData.R:81:20:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.readData.R#L81) *style:* **Commas should always have a space after.**
```r
  res <- reader(fd,...)
                   ^
```
[R/exa.writeData.R:36:42:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L36) *style:* **Variable and function names should be all lowercase.**
```r
exa.writeData <- function(channel, data, tableName, tableColumns = NA,
                                         ^~~~~~~~~
```
[R/exa.writeData.R:36:53:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L36) *style:* **Variable and function names should be all lowercase.**
```r
exa.writeData <- function(channel, data, tableName, tableColumns = NA,
                                                    ^~~~~~~~~~~~
```
[R/exa.writeData.R:37:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L37) *style:* **lines should not be more than 80 characters.**
```r
                          encoding = tryCatch(strsplit(Sys.getlocale("LC_CTYPE"),".", fixed=TRUE)[[1]][2],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.writeData.R:37:82:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L37) *style:* **Commas should always have a space after.**
```r
                          encoding = tryCatch(strsplit(Sys.getlocale("LC_CTYPE"),".", fixed=TRUE)[[1]][2],
                                                                                ^
```
[R/exa.writeData.R:37:92:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L37) *style:* **Put spaces around all infix operators.**
```r
                          encoding = tryCatch(strsplit(Sys.getlocale("LC_CTYPE"),".", fixed=TRUE)[[1]][2],
                                                                                ~^~
```
[R/exa.writeData.R:38:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L38) *style:* **lines should not be more than 80 characters.**
```r
                                              error = function(e) stop(paste("Cannot get system encoding.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.writeData.R:39:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L39) *style:* **lines should not be more than 80 characters.**
```r
                                                                             Please set manually.\n",e)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.writeData.R:39:102:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L39) *style:* **Commas should always have a space after.**
```r
                                                                             Please set manually.\n",e)
                                                                                ^
```
[R/exa.writeData.R:42:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L42) *style:* **lines should not be more than 80 characters.**
```r
                          writer = function(data, conn, fileEncoding = encoding) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.writeData.R:42:57:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L42) *style:* **Variable and function names should be all lowercase.**
```r
                          writer = function(data, conn, fileEncoding = encoding) {
                                                        ^~~~~~~~~~~~
```
[R/exa.writeData.R:50:56:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L50) *style:* **Variable and function names should be all lowercase.**
```r
                                        fileEncoding = fileEncoding)
                                                       ^~~~~~~~~~~~
```
[R/exa.writeData.R:58:15:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L58) *style:* **Variable and function names should be all lowercase.**
```r
    server <- odbcGetInfo(channel)[["Server_Name"]]
              ^~~~~~~~~~~
```
[R/exa.writeData.R:61:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L61) *style:* **Variable and function names should be all lowercase.**
```r
  serverAddress <- strsplit(server, ":")[[1]]
  ^~~~~~~~~~~~~
```
[R/exa.writeData.R:63:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  serverHost <- as.character(serverAddress[[1]])
  ^~~~~~~~~~
```
[R/exa.writeData.R:63:30:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  serverHost <- as.character(serverAddress[[1]])
                             ^~~~~~~~~~~~~
```
[R/exa.writeData.R:64:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L64) *style:* **Variable and function names should be all lowercase.**
```r
  serverPort <- as.integer(serverAddress[[2]])
  ^~~~~~~~~~
```
[R/exa.writeData.R:64:28:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L64) *style:* **Variable and function names should be all lowercase.**
```r
  serverPort <- as.integer(serverAddress[[2]])
                           ^~~~~~~~~~~~~
```
[R/exa.writeData.R:66:36:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L66) *style:* **Variable and function names should be all lowercase.**
```r
  .Call(C_asyncRODBCIOStart, slot, serverHost, serverPort)
                                   ^~~~~~~~~~
```
[R/exa.writeData.R:66:48:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L66) *style:* **Variable and function names should be all lowercase.**
```r
  .Call(C_asyncRODBCIOStart, slot, serverHost, serverPort)
                                               ^~~~~~~~~~
```
[R/exa.writeData.R:67:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L67) *style:* **Variable and function names should be all lowercase.**
```r
  proxyHost <- .Call(C_asyncRODBCProxyHost, slot)
  ^~~~~~~~~
```
[R/exa.writeData.R:68:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L68) *style:* **Variable and function names should be all lowercase.**
```r
  proxyPort <- .Call(C_asyncRODBCProxyPort, slot)
  ^~~~~~~~~
```
[R/exa.writeData.R:70:35:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L70) *style:* **Variable and function names should be all lowercase.**
```r
  query <- paste0("IMPORT INTO ", tableName,
                                  ^~~~~~~~~
```
[R/exa.writeData.R:71:28:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L71) *style:* **Variable and function names should be all lowercase.**
```r
                 if (is.na(tableColumns)) ""
                           ^~~~~~~~~~~~
```
[R/exa.writeData.R:72:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                      ^
```
[R/exa.writeData.R:72:34:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Commas should always have a space after.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                 ^
```
[R/exa.writeData.R:72:40:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Variable and function names should be all lowercase.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                       ^~~~~~~~~~~~
```
[R/exa.writeData.R:72:53:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Commas should always have a space after.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                                    ^
```
[R/exa.writeData.R:72:61:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Put spaces around all infix operators.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                                           ~^~
```
[R/exa.writeData.R:72:68:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Commas should always have a space after.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                                                   ^
```
[R/exa.writeData.R:72:72:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L72) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
                 else {paste("(",paste(tableColumns,collapse=", "),")")},
                                                                       ^
```
[R/exa.writeData.R:73:43:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L73) *style:* **Variable and function names should be all lowercase.**
```r
                 " FROM CSV AT 'http://", proxyHost, ":",
                                          ^~~~~~~~~
```
[R/exa.writeData.R:74:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L74) *style:* **lines should not be more than 80 characters.**
```r
                 proxyPort, "' FILE 'importData.csv' ENCODING = '", encoding, "'")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/exa.writeData.R:74:18:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/exa.writeData.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
                 proxyPort, "' FILE 'importData.csv' ENCODING = '", encoding, "'")
                 ^~~~~~~~~
```
[R/EXADBI-internal.R:2:2:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:67:9:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L67) *style:* **Commented code should be removed.**
```r
#       return(sql("")) # empty string is NULL in EXASOL
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI-internal.R:77:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L77) *style:* **Commas should always have a space after.**
```r
      if (substr(x, 1,1) == "\"" & substring(x, nchar(x)) == "\"")
                      ^
```
[R/EXADBI-internal.R:84:14:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L84) *style:* **Only use double-quotes.**
```r
  id <- gsub('.', '_', id, fixed = TRUE)# change dots to underscores
             ^~~
```
[R/EXADBI-internal.R:84:19:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L84) *style:* **Only use double-quotes.**
```r
  id <- gsub('.', '_', id, fixed = TRUE)# change dots to underscores
                  ^~~
```
[R/EXADBI-internal.R:87:17:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L87) *style:* **Commas should always have a space after.**
```r
  paste0(quotes,make.unique(encodeString(id)),quotes)
                ^
```
[R/EXADBI-internal.R:87:47:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L87) *style:* **Commas should always have a space after.**
```r
  paste0(quotes,make.unique(encodeString(id)),quotes)
                                              ^
```
[R/EXADBI-internal.R:107:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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:157:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI-internal.R#L161) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/EXADBI.R:10:2:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/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/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L64) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:102:5:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L102) *style:* **Commented code should be removed.**
```r
  # path.package("exasol")
    ^~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:105:5:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L105) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(driver)) {
    ^
```
[R/EXADBI.R:109:20:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L109) *style:* **Only use double-quotes.**
```r
        Sys.info()['sysname'],
                   ^~~~~~~~~
```
[R/EXADBI.R:110:9:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L110) *style:* **Only use double-quotes.**
```r
        'Darwin' = paste0(system.file(package = packageName()),
        ^~~~~~~~
```
[R/EXADBI.R:111:27:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L111) *style:* **Only use double-quotes.**
```r
                          'odbc/lib/darwin/x86_64/libexaodbc-io418sys.dylib'
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:113:9:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L113) *style:* **Only use double-quotes.**
```r
        'Linux' = paste0(system.file(package = packageName()),
        ^~~~~~~
```
[R/EXADBI.R:114:26:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L114) *style:* **Only use double-quotes.**
```r
                         'odbc/lib/linux/x86_64/libexaodbc-uo2214lv1.so'
                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:119:21:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L119) *style:* **Put spaces around all infix operators.**
```r
  } else if (driver =="SYSTEM") {
                    ^~~
```
[R/EXADBI.R:120:7:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L120) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!silent) print("Using the system driver...")
      ^
```
[R/EXADBI.R:127:3:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L127) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver", odbc_drv = driver)
  ^~~
```
[R/EXADBI.R:137:44:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L137) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:145:25:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L145) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:204:15:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L204) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAConnection",
              ^
```
[R/EXADBI.R:205:25:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L205) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:206:10:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L206) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
         ^~~~~~~~~
```
[R/EXADBI.R:206:20:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L206) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
                   ^~~~~
```
[R/EXADBI.R:210:26:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L210) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                         ^~~~~
```
[R/EXADBI.R:210:46:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L210) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                                             ^~~~~
```
[R/EXADBI.R:211:16:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L211) *style:* **Variable and function names should be all lowercase.**
```r
      dbname = dbObj@db_name,
               ^~~~~
```
[R/EXADBI.R:212:18:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L212) *style:* **Variable and function names should be all lowercase.**
```r
      username = dbObj@db_user,
                 ^~~~~
```
[R/EXADBI.R:213:14:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L213) *style:* **Variable and function names should be all lowercase.**
```r
      host = dbObj@db_host,
             ^~~~~
```
[R/EXADBI.R:214:14:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      port = dbObj@db_port
             ^~~~~
```
[R/EXADBI.R:223:44:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L223) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:274:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L274) *style:* **lines should not be more than 80 characters.**
```r
    #         close = function(commit=TRUE) {                         # dbClearResult is in row 880
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:275:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L275) *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:278:21:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L278) *style:* **Commented code should be removed.**
```r
    #               warning("Connection seems exipired.\n    ...it's gone...")
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:279:21:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L279) *style:* **Commented code should be removed.**
```r
    #               return(FALSE)
                    ^~~~~~~~~~~~~
```
[R/EXADBI.R:283:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L283) *style:* **Commented code should be removed.**
```r
    #                 if(commit) message("Changes commited.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:284:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L284) *style:* **lines should not be more than 80 characters.**
```r
    #             } else stop("Commit failed. Changes NOT commited. Closing aborted.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:285:17:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L285) *style:* **Commented code should be removed.**
```r
    #           message("Closing connection...")
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:286:19:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L286) *style:* **Commented code should be removed.**
```r
    #             res <- try(odbcClose(connection),silent=TRUE)
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:288:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L288) *style:* **Commented code should be removed.**
```r
    #                 message("Connection closed.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:289:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L289) *style:* **Commented code should be removed.**
```r
    #                 return(TRUE)
                      ^~~~~~~~~~~~
```
[R/EXADBI.R:291:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L291) *style:* **Commented code should be removed.**
```r
    #                 warning("Closing not successful.")
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:292:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L292) *style:* **Commented code should be removed.**
```r
    #                 return(FALSE)
                      ^~~~~~~~~~~~~
```
[R/EXADBI.R:294:27:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L294) *style:* **Commented code should be removed.**
```r
    #                     warning(res)
                          ^~~~~~~~~~~~
```
[R/EXADBI.R:295:27:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L295) *style:* **Commented code should be removed.**
```r
    #                     return(FALSE)
                          ^~~~~~~~~~~~~
```
[R/EXADBI.R:300:8:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L300) *style:* **Commented code should be removed.**
```r
      #close()
       ^~~~~~~
```
[R/EXADBI.R:307:15:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L307) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAResult",
              ^
```
[R/EXADBI.R:308:25:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L308) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:309:7:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L309) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(dbObj$temp_result_tbl == "CLEARED") stop("GetInfo: Result set cleared.")
      ^
```
[R/EXADBI.R:309:8:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L309) *style:* **Variable and function names should be all lowercase.**
```r
    if(dbObj$temp_result_tbl == "CLEARED") stop("GetInfo: Result set cleared.")
       ^~~~~
```
[R/EXADBI.R:311:19:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L311) *style:* **Variable and function names should be all lowercase.**
```r
      statement = dbObj$statement,
                  ^~~~~
```
[R/EXADBI.R:312:19:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L312) *style:* **Variable and function names should be all lowercase.**
```r
      row.count = dbObj$rows_fetched,
                  ^~~~~
```
[R/EXADBI.R:313:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L313) *style:* **Variable and function names should be all lowercase.**
```r
      rows.affected = dbObj$rows_affected,
                      ^~~~~
```
[R/EXADBI.R:314:23:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L314) *style:* **Variable and function names should be all lowercase.**
```r
      has.completed = dbObj$is_complete,
                      ^~~~~
```
[R/EXADBI.R:315:19:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L315) *style:* **Variable and function names should be all lowercase.**
```r
      is.select = dbObj$with_output
                  ^~~~~
```
[R/EXADBI.R:323:44:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L323) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:336:25:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L336) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:347:47:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L347) *style:* **Variable and function names should be all lowercase.**
```r
setMethod("dbDataType", "EXAObject", function(dbObj, obj, ...) {
                                              ^~~~~
```
[R/EXADBI.R:348:5:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L348) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(any(class(obj) %in% c("factor", "data.frame")))
    ^
```
[R/EXADBI.R:382:2:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L382) *style:* **Commented code should be removed.**
```r
#setOldClass("AsIs")
 ^~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:383:2:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L383) *style:* **Commented code should be removed.**
```r
#setMethod("EXADataType", "ANY",  definition=function(x) EXADataType(unclass(x)))
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:395:5:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L395) *style:* **Variable and function names should be all lowercase.**
```r
    dbGetInfo(drv, "connectionIds")[[1]]
    ^~~~~~~~~
```
[R/EXADBI.R:459:1:](https://github.com/marcelboldt/r-exasol/blob/5ef5a3dc9a18acd89cebaeb91c19bdb6e676b70a/R/EXADBI.R#L459) *style:* **lines should not be more than 80 characters.**
```r
                        exalogfile = tempfile(pattern = "EXAODBC_", fileext = ".log"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


On 04/29/2016 03:12 PM, Jürgen Fischer wrote:
> what about 8dcd918?

Thanks for the pointer, I overlooked that one.


Making the string_to_compound function a bit more rigorous by design.


Such a long regex was not necessary.


After looking at code coverage reports, this line was never covered and is unnecessary. Elements will never be null if it passed the previous tests.